Description

This PR is a follow-up to PR #277 which added submenu icon support across all platforms. While the initial implementation worked on macOS and Linux, I discovered that submenu icons were not displaying correctly on Windows due to incorrect ID handling.

Problem

On Windows, submenu icons were not displaying correctly. The issue was that the Windows API requires different handling for submenu icons compared to regular menu items.

Root Cause

In src/platform_impl/windows/mod.rs, the code was using the same ID (internal_id) for both regular menu items and submenus when setting icons:

SetMenuItemInfoW(self.hmenu, child_.internal_id(), FALSE, &info);

However, Windows API requires different IDs for submenus (hmenu) vs regular menu items (internal_id).

Solution

Modified the icon setting logic to use the correct ID based on the menu item type:

1. For regular menu items: Use internal_id as before
2. For submenus: Use hmenu as the ID

Updated both the initial icon setting in add_menu_item and the dynamic icon setting in set_icon method.

Changes Made

1. src/platform_impl/windows/mod.rs:

   • Modified icon setting logic to use correct IDs for submenus vs regular items
   • Updated both add_menu_item and set_icon methods
   • Added proper handling for submenu icon creation

2. Examples:

   • Added submenu icon examples in examples/test_submenu_icons.rs
   • Updated existing examples to demonstrate submenu icon functionality

Testing

• Tested on Windows with submenu icons
• Verified that both text and icons display correctly
• Confirmed compatibility with existing functionality
• Added example demonstrating submenu icon usage